Skip to content

docs: clarify when LLM API keys are required vs optional#1386

Closed
Anujpandey12345 wants to merge 3 commits intoaden-hive:mainfrom
Anujpandey12345:docs/clarify-llm-api-keys
Closed

docs: clarify when LLM API keys are required vs optional#1386
Anujpandey12345 wants to merge 3 commits intoaden-hive:mainfrom
Anujpandey12345:docs/clarify-llm-api-keys

Conversation

@Anujpandey12345
Copy link

This PR clarifies when LLM API keys are required versus optional for contributors.

While setting up the project locally, I noticed that contributors without LLM credits may assume they cannot contribute, even though many workflows (documentation, tooling, framework improvements) do not require keys.

This change adds a short section under Quick Start → Prerequisites to make this distinction explicit and improve onboarding clarity.

Closes #1033

@Anujpandey12345
Copy link
Author

Thanks! Happy to make any adjustments if needed. Looking forward to your review.

Copy link
Collaborator

@bryanadenhq bryanadenhq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks great, thanks for contributing!

@bryanadenhq
Copy link
Collaborator

Hi @Anujpandey12345, the seems to be a linter error. Could you please run ruff check . and fix the issues? Thanks!

@Anujpandey12345
Copy link
Author

@bryanadenhq Thanks for the heads up! I’ve run ruff locally and pushed the fixes. Please let me know if anything else is needed.

@bryanadenhq
Copy link
Collaborator

Hi @Anujpandey12345, it seems like there was no update that was pushed?

@Anujpandey12345
Copy link
Author

@bryanadenhq Thanks for the patience! I’ve pushed a new commit after running ruff check . locally to re-trigger CI. Please let me know if anything else is needed.

@Anujpandey12345
Copy link
Author

@bryanadenhq Any update?

Copy link
Collaborator

@austin931114 austin931114 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

close for outdated doc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] Clarify when LLM API keys are required vs optional for contributors

3 participants